Skip to content

Notification status#215

Open
Adeswalla wants to merge 5 commits into
Core-Foundry:mainfrom
Adeswalla:notification-status
Open

Notification status#215
Adeswalla wants to merge 5 commits into
Core-Foundry:mainfrom
Adeswalla:notification-status

Conversation

@Adeswalla

Copy link
Copy Markdown
Contributor

Closes #198

@drips-wave

drips-wave Bot commented Jun 24, 2026

Copy link
Copy Markdown

@Adeswalla Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

- dashboard: raise jest testTimeout to 15s to prevent flaky timeouts
  when render-benchmark.test runs in parallel with async userEvent tests
- dashboard: fix stale-cache deduplication (Map last-write-wins, correct
  appendEvents merge order) and add regression tests
- dashboard: add 15s polling to EventExplorerPage and EventsPage so
  status updates appear without a manual refresh
- listener: fix notification-scheduler-refactored test — fetchAndLock
  was picking up both past notifications; restore item 2 to PENDING
  so only item 3 is left as PROCESSING with an expired lock
- listener: remove accidentally committed debug-test.db; add data/ to
  .gitignore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Fix Delayed Notification Status Synchronization

1 participant